Skip to content

[BUILD] Upgrade Maven to 3.9.15#12059

Open
Yao-MR wants to merge 2 commits into
apache:mainfrom
Yao-MR:enhancement/enable_mvn3915
Open

[BUILD] Upgrade Maven to 3.9.15#12059
Yao-MR wants to merge 2 commits into
apache:mainfrom
Yao-MR:enhancement/enable_mvn3915

Conversation

@Yao-MR
Copy link
Copy Markdown
Contributor

@Yao-MR Yao-MR commented May 9, 2026

What changes are proposed in this pull request?

Bump maven version to 3.9.15

Align with upstream:
apache/spark#55398 - Bump Maven to 3.9.15
apache/hadoop#8479 - Bump Maven to 3.9.15

Maven 3.9.15 (2025-10-24)
Bug Fixes

  • [MNG-8742] Fix module ordering error when --resume-from and --projects are used together
  • [MNG-8755] Fix mvnw wrapper handling of paths with spaces on Windows
  • [MNG-8760] NPE in MavenProject.getCompileClasspathElements() on Java 24
  • [MNG-8768] Regression in maven-compiler-plugin interaction with the --release flag

Improvements

  • [MNG-8735] Improved -X debug output to reduce leakage of sensitive info (passwords, tokens)
  • [MNG-8748] maven-resolver HTTP connection pool default tuning (~5% performance gain)

Dependency Updates

  • maven-resolver 1.9.23 → 1.9.24
  • guava 33.3.1-jre → 33.4.0-jre
  • commons-lang3 3.17.0 → 3.18.0

How was this patch tested?

Pass CI.

Was this patch authored or co-authored using generative AI tooling?

@github-actions github-actions Bot added CORE works for Gluten Core BUILD labels May 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

Run Gluten Clickhouse CI on x86

@Yao-MR Yao-MR marked this pull request as ready for review May 9, 2026 06:07
@Yao-MR
Copy link
Copy Markdown
Contributor Author

Yao-MR commented May 11, 2026

hi @philo-he could you help make a review when you have a chance?

@philo-he
Copy link
Copy Markdown
Member

@Yao-MR
Copy link
Copy Markdown
Contributor Author

Yao-MR commented May 14, 2026

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@Yao-MR
Copy link
Copy Markdown
Contributor Author

Yao-MR commented May 20, 2026

The following code may also need updating. Could you check if there are other places that need changes as well? Thank you.
https://github.com/philo-he/gluten/blob/6fa6062a92f6dbd320a77ab6075f67f5d92ba97e/dev/docker/Dockerfile.centos8-dynamic-build#L35
https://github.com/philo-he/gluten/blob/6fa6062a92f6dbd320a77ab6075f67f5d92ba97e/.github/workflows/util/setup-helper.sh#L21

sure,will overall check.

Thanks @philo-he for the careful review. I went through the whole repo and here's the status.

Already updated in this PR:

  • pom.xml: <maven.version> 3.9.13 -> 3.9.15
  • dev/vcpkg/setup-build-depends.sh: maven_version -> 3.9.15

About the two files you pointed out:
The links reference my fork at an older commit. After rebasing onto the latest main:

  • dev/docker/Dockerfile.centos8-dynamic-build no longer installs apache-maven-*-bin.tar.gz directly; it relies on the build/mvn wrapper, so nothing to change there.
  • .github/workflows/util/setup-helper.sh no longer exists on main.

Other Maven 3.6.3 references I found but intentionally left unchanged:

  1. tools/gluten-te/ubuntu/dockerfile-build and tools/gluten-te/centos/dockerfile-build
    The apache-maven-3.6.3 download is gated behind JDK_DEBUG_BUILD=ON (a custom OpenJDK 8 fastdebug image) and the build is hard-pinned to -Pspark-3.5. It does not participate in the Spark 4.1 build path, so bumping it would expand the scope of this PR without any functional benefit.

  2. 3.6.3 in pom.xml and package/pom.xml
    This matches upstream Spark ([SPARK-56530][BUILD] Upgrade Maven to 3.9.15 spark#55398): only <maven.version> (the build/mvn wrapper hint) is bumped, while the enforcer floor is kept so users still building Spark 3.3 / 3.4 / 3.5 with Maven 3.6.x or 3.8.x are not broken. Spark 4.1's hard requirement is JDK 17 + Scala 2.13, not a Maven minimum.

In short, the scope of this PR only covers the places Spark 4.1 actually requires; the legacy Spark 3.x build paths keep their existing Maven floor for backward compatibility, following the same approach as upstream Spark.

Happy to bump the remaining references to 3.9.15 if you'd prefer full unification. Let me know your preference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BUILD CORE works for Gluten Core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants